Skip to content

Review CLAUDE.md and update Open Spec documentation#44

Merged
watterfall merged 1 commit intomainfrom
claude/review-claude-open-spec-01G4QxgrBP48VhfXVRzpBuzH
Nov 22, 2025
Merged

Review CLAUDE.md and update Open Spec documentation#44
watterfall merged 1 commit intomainfrom
claude/review-claude-open-spec-01G4QxgrBP48VhfXVRzpBuzH

Conversation

@watterfall
Copy link
Contributor

… issues

  • Add Documentation Architecture section (Section 0) to OPENSPEC.md v1.2.0

    • Define clear responsibilities: CLAUDE.md for AI assistant, OPENSPEC.md for developers
    • Document update principles to avoid duplication
  • Remove 'use client' directive from 9 cyber React components

    • Eliminate sourcemap warnings during build
    • 'use client' is Next.js syntax, unnecessary in Astro projects
  • Add Props interface to core widget components

    • Hero.astro, Vision.astro, CallToAction.astro now have typed Props
    • Follows OPENSPEC component standards
  • Update ROADMAP.md v1.4.0 with completed tasks

    • Build status: 0 errors, 0 warnings, 1 hint

… issues

- Add Documentation Architecture section (Section 0) to OPENSPEC.md v1.2.0
  - Define clear responsibilities: CLAUDE.md for AI assistant, OPENSPEC.md for developers
  - Document update principles to avoid duplication

- Remove 'use client' directive from 9 cyber React components
  - Eliminate sourcemap warnings during build
  - 'use client' is Next.js syntax, unnecessary in Astro projects

- Add Props interface to core widget components
  - Hero.astro, Vision.astro, CallToAction.astro now have typed Props
  - Follows OPENSPEC component standards

- Update ROADMAP.md v1.4.0 with completed tasks
  - Build status: 0 errors, 0 warnings, 1 hint
@vercel
Copy link

vercel bot commented Nov 22, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
website Ready Ready Preview Comment Nov 22, 2025 5:46pm

@watterfall watterfall merged commit 9325044 into main Nov 22, 2025
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants